Feature/nginx version bump#10
Conversation
There was a problem hiding this comment.
Pull request overview
Bumps the stack’s version and updates the default configured Nginx version used by the Roll configuration schema.
Changes:
- Incremented the project version from
0.7.0.1to0.7.0.2. - Updated
NGINX_VERSIONdefault in the config schema from1.24to1.27.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
version |
Updates the overall stack release/version number. |
utils/config.sh |
Updates the config schema default for NGINX_VERSION. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ROLL_CONFIG_SCHEMA_KEYS+=(OPENSEARCH_VERSION); ROLL_CONFIG_SCHEMA_VALUES+=("string:2.5") | ||
| ROLL_CONFIG_SCHEMA_KEYS+=(MONGO_VERSION); ROLL_CONFIG_SCHEMA_VALUES+=("string:6.0") | ||
| ROLL_CONFIG_SCHEMA_KEYS+=(NGINX_VERSION); ROLL_CONFIG_SCHEMA_VALUES+=("string:1.24") | ||
| ROLL_CONFIG_SCHEMA_KEYS+=(NGINX_VERSION); ROLL_CONFIG_SCHEMA_VALUES+=("string:1.27") |
There was a problem hiding this comment.
The default NGINX_VERSION was bumped here to 1.27, but the docker-compose template still falls back to 1.26 when NGINX_VERSION is unset (see environments/includes/nginx.base.yml). This creates inconsistent defaults depending on whether users go through the CLI/config loader or run compose directly. Please update the compose fallback (or remove the fallback and rely on the exported config) so the default version is consistent across the stack.
No description provided.